Skip to content

fix: sanitize GitHub username inputs - #255

Merged
O2sa merged 1 commit into
O2sa:mainfrom
abhinavrastogi2627:fix/sanitize-github-usernames
Oct 20, 2026
Merged

O2sa merged 1 commit into
O2sa:mainfrom
abhinavrastogi2627:fix/sanitize-github-usernames

Conversation

@abhinavrastogi2627

Copy link
Copy Markdown
Contributor

Fixes #254

Sanitizes leading @ characters, GitHub profile URLs, surrounding whitespace, and trailing slashes before comparison and profile requests reach GitHub.

Coverage includes the shared helper, comparison form, comparison API, and profile API.

Validation performed:

  • ./node_modules/.bin/vitest run — 29 files, 237 tests passed
  • ./node_modules/.bin/tsc --noEmit — passed
  • ./node_modules/.bin/eslint . — no errors; one existing no-img-element warning in leaderboard-table.test.tsx
  • ./node_modules/.bin/prettier --check . — passed
  • git diff --check — passed

The issue is open and unblocked at submission.

@vercel

vercel Bot commented Oct 20, 2026

Copy link
Copy Markdown

@abhinavrastogi2627 is attempting to deploy a commit to the osama's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thank you for the pull request! ✅

A maintainer will review this soon. Please be patient while we take a look. 🙌

@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
dev-impact Ready Ready Preview Oct 20, 2026 7:16pm UTC

@O2sa
O2sa merged commit f66f1c0 into O2sa:main Oct 20, 2026
4 checks passed
@O2sa

O2sa commented Oct 20, 2026

Copy link
Copy Markdown
Owner

✨ Thank you, @abhinavrastogi2627! Another great contribution merged! 🚀

We truly appreciate your continued support! If you find this tool useful, please consider giving us a ⭐ star on GitHub—it helps more developers find our work and motivates us to keep improving!

This branch was successfully deployed

1 active deployment
Preview — d77c6f53 Deployed Oct 20, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

enhancement: Sanitize developer usernames by stripping leading '@' and GitHub URLs

2 participants